Skip to content

fix(contacts): label app content regions#5375

Open
hamza-agent wants to merge 1 commit into
nextcloud:mainfrom
hamza-agent:fix/main-content-aria-labels
Open

fix(contacts): label app content regions#5375
hamza-agent wants to merge 1 commit into
nextcloud:mainfrom
hamza-agent:fix/main-content-aria-labels

Conversation

@hamza-agent

Copy link
Copy Markdown

Summary

  • add accessible labels to the contacts main content states
  • add accessible labels to the teams and organization chart app content regions

Testing

  • npm run lint (passes with existing warnings)

Fixes #5046

Add accessible labels to the contacts, teams, and organization chart main content containers so screen readers can identify the active app region.

Fixes nextcloud#5046

Assisted-by: agent-hamza:gpt-5
Signed-off-by: 00hamza <291504006+hamza-agent@users.noreply.github.com>
@hamza221 hamza221 added the 3. to review Waiting for reviews label Jun 8, 2026
@hamza221

hamza221 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

@hamza-agent what is this PR about ?

@codecov

codecov Bot commented Jun 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/components/AppContent/ContactsContent.vue 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@hamza-agent

Copy link
Copy Markdown
Author

It's a small accessibility fix for the contacts app content regions. The main content areas currently change between states like the contact list, teams view, organization chart, and empty/loading views, but some of those regions do not expose a clear accessible label. This PR adds/normalizes those labels so screen reader users can identify the current app content region more reliably.

No behavior change intended beyond the ARIA labels.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing aria label identifier for main view

2 participants